Skip to content

Fluentd / FluentBit out_forward compat#9

Merged
jsvd merged 2 commits into
logstash-plugins:masterfrom
tes:fluentd-compat
Jan 27, 2017
Merged

Fluentd / FluentBit out_forward compat#9
jsvd merged 2 commits into
logstash-plugins:masterfrom
tes:fluentd-compat

Conversation

@MerlinDMC

Copy link
Copy Markdown
Contributor

This does add compatibility with later Fluentd and FluentBit out_forward encodings.
Both out_forward streams are sending combined packets that bundle up multiple events for a single tag.

This currently does not add support for compressed streams. It will raise a failure and log it in the event stream.

This should also fix #4 and #2

@krallistic

Copy link
Copy Markdown

Any chance that this will get merged?

@joeris-CGM

joeris-CGM commented Jan 27, 2017

Copy link
Copy Markdown

+1, merge please

@jsvd jsvd merged commit 6734f1c into logstash-plugins:master Jan 27, 2017
@agonzalezro

Copy link
Copy Markdown

Could this be released as a gem? I bit the same error and the only solution I found was to use copy the file in master to the logstash container.

It seems that logstash is always using the latest version (I couldn't find a Gemfile.lock): https://github.com/elastic/logstash/blob/master/Gemfile#L36

In case you couldn't release, what would be the way to move this from my test environment to production? Override the gem in logstash with the master of this repo?

Thanks!

@ThoTischner

Copy link
Copy Markdown

+1 Please release this fix.

@jordansissel

Copy link
Copy Markdown
Contributor

I am attempting to do a release now.

@jordansissel

Copy link
Copy Markdown
Contributor

Version 3.1.1 of this plugin is now published.

@agonzalezro

agonzalezro commented Mar 15, 2017

Copy link
Copy Markdown

Awesome! Thanks @jordansissel

@hselvara

Copy link
Copy Markdown

Hi All,

I have a made a pull request for the avoiding overwriting of tags with key named as 'tags' with #10 .Any chance this will get merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Codec kills listener on exception

8 participants